Java05. ДЗ 04, Кравцун Андрей by kravtsun · Pull Request #66 · java-course-au/assignments-2017-1

kravtsun · 2017-04-06T22:52:36Z

…AYS_FALSE).

sproshev · 2017-04-10T19:51:42Z

src/main/java/ru/spbau/mit/Collections.java

+import java.util.ArrayList;
+import java.util.Collection;
+
+public final class Collections {


если у класса только 1 конструктор и тот приватный, то final можно не писать

Checkstyle 6.11.2 жалуется конкретно на это. Исправлять?

sproshev · 2017-04-10T19:52:50Z

src/main/java/ru/spbau/mit/Collections.java

+public final class Collections {
+    private Collections() {}
+
+    public static <T, S> Collection<S> map(Function1<? super T, S> f, Collection<T> c) {


см задание, там указаны другие принимаемые и возвращаемые типы

Исправил, насколько понял условие.

sproshev · 2017-04-10T19:55:26Z

src/main/java/ru/spbau/mit/Function2.java

@@ -0,0 +1,42 @@
+package ru.spbau.mit;
+
+// extends Function1<T1, Function1<T2, S>>


не надо)

sproshev · 2017-04-10T19:56:00Z

src/main/java/ru/spbau/mit/Predicate.java

+package ru.spbau.mit;
+
+public abstract class Predicate<T> extends Function1<T, Boolean> {
+    public static final Predicate ALWAYS_TRUE = new Predicate() {


generic-параметры?

Generic-параметры к классу? к оператору new (извиняюсь заранее, если дико ошибаюсь по фактике)? Или хорошим тоном считается всегда специфицировать параметр у параметризуемого типа, навроде "public static final Predicate ALWAYS_TRUE = new Predicate()"?

generic параметр к типу константы

не только хорошим тоном, но и обязательным )

если компилятор его не вывел, если вывел, то указывать не надо

Тогда странно, что после выставления половина тестов перестала компилироваться.

как теперь эта строка выглядит?

public static final Predicate ALWAYS_TRUE = new Predicate() {...};
Так нормально? Просто я не могу привыкнуть к мысли, что все бинарные операции несимметричны, поэтому, например, ALWAYS_TRUE нельзя ставить в левую часть бинарного отношения с FAIL_INTEGER.

sproshev · 2017-04-10T19:56:07Z

src/main/java/ru/spbau/mit/Predicate.java

+        }
+    };
+
+    public static final Predicate ALWAYS_FALSE = new Predicate() {


и здесь

Прошу пояснения, мы должны давать возможность кастовать Predicate на обобщение T?

не совсем, надо явно указать, что ALWAYS_FALSE готов принимать любые объекты, при этом он должен быть параметризован

sproshev · 2017-04-10T19:59:08Z

src/test/java/ru/spbau/mit/CollectionsTest.java

+    private static final Integer MAGIC_2 = -4;
+    private static final Integer MAGIC_3 = 3;
+    private static final Integer MAGIC_4 = 10;
+    private static final Integer MAGIC_5 = -20;


лучше отключить проверку на константы в тестах

и не использовать обертки без надобности

checkstyle.xml находится не в моем репозитории. Был бы рад отключить проверку на константы, если бы знал, что сделаю это всего один раз.

sproshev · 2017-04-10T19:59:14Z

src/test/java/ru/spbau/mit/CollectionsTest.java

+    private static final Integer MAGIC_4 = 10;
+    private static final Integer MAGIC_5 = -20;
+
+    private static final ArrayList<Integer> INT_ARR = new ArrayList<Integer>() { {


Arrays.asList(...)

sproshev · 2017-04-10T19:59:46Z

src/test/java/ru/spbau/mit/CollectionsTest.java

+        }
+    };
+
+    private static final List<Object> EMPTY_ARR = new ArrayList<>();


Collections.emptyList()

sproshev · 2017-04-10T20:01:12Z

src/test/java/ru/spbau/mit/CollectionsTest.java

+        assertEquals(0, Collections.foldr(FAIL_FUNCTION, EMPTY_ARR, 0));
+
+    }
+


wildcardы надо проверять

Вставил пару тестов. Что конкретно необходимо проверить?

sproshev · 2017-04-10T20:04:00Z

src/test/java/ru/spbau/mit/PredicateTest.java

+    private static final Predicate<Double> EQUAL_TO_ONE = new Predicate<Double>() {
+        @Override
+        public Boolean apply(Double arg) {
+            return ONE_DOUBLE == arg;


нельзя, это объекты, лучше Objects.equals(ONE_DOUBLE, arg). тот сперва проверит на ==, потом вызовет O_D.equals(arg)

-1

Исправил.

sproshev

8.5

sproshev · 2017-05-05T18:23:11Z

src/main/java/ru/spbau/mit/Predicate.java

+    }
+
+    @Override
+    public abstract Boolean apply(T arg);


это не требуется, мы же его унаследуем от Function1<T, Boolean>

sproshev · 2017-05-05T18:25:24Z

src/test/java/ru/spbau/mit/CollectionsTest.java

+
+    private static final Predicate<Object> NOT_NULL = IS_NULL.not();
+
+    private static final List<Object> EMPTY_ARR = (List<Object>) emptyList();


каст ни к чему

sproshev · 2017-05-05T18:26:11Z

src/test/java/ru/spbau/mit/CollectionsTest.java

+        final ArrayList<Integer> positiveOnly = new ArrayList<Integer>() { {
+            add(-MAGIC_2);
+            add(-MAGIC_5);
+        } };


Arrays.asList

sproshev · 2017-05-05T18:27:33Z

src/test/java/ru/spbau/mit/CollectionsTest.java

+            }
+        };
+        final int ten = 10;
+        assertEquals(Arrays.asList(ten), Collections.filter(isTen, INT_ARR));


если хочется создать список из 1 элемента, лучше использовать Collections.singletonList(...), так не будет создаваться массив, который будет передан в качестве варарга в Arrays.asList

sproshev · 2017-05-05T18:28:10Z

src/test/java/ru/spbau/mit/CollectionsTest.java

+        assertEquals(EMPTY_ARR, Collections.takeUnless(IS_POSITIVE_INTEGER.not(), INT_ARR_NEGATED));
+        assertEquals(Arrays.asList(-1), Collections.takeUnless(IS_POSITIVE_INTEGER, INT_ARR_NEGATED));
+        assertEquals(INT_ARR, Collections.takeUnless(Predicate.ALWAYS_FALSE, INT_ARR));
+        ArrayList<Objects> nullArr = new ArrayList<>();


<Integer>

sproshev · 2017-05-05T18:28:46Z

src/test/java/ru/spbau/mit/CollectionsTest.java

+    @Test
+    public void foldl() throws Exception {
+        final Integer result = 10;
+        assertEquals(result, Collections.foldl(MINUS, INT_ARR, (Integer) 0));


каст литерала к обертке не требуется, это выполнится автоматически

sproshev · 2017-05-05T18:29:36Z

src/test/java/ru/spbau/mit/Function1Test.java

+import static org.junit.Assert.*;
+
+public class Function1Test {
+    private static Function1<Integer[], Collection<Integer>> fromArrayToCollection =


final, caps

sproshev · 2017-05-05T18:30:12Z

src/test/java/ru/spbau/mit/Function1Test.java

+                    for (Object o : arg) {
+                        c.add((Integer) o);
+                    }
+                    return c;


return Arrays.asList(arg);

sproshev · 2017-05-05T18:30:20Z

src/test/java/ru/spbau/mit/Function1Test.java

+        }
+    };
+
+    private final Function2<Integer, Integer, Integer> minus = new Function2<Integer, Integer, Integer>() {


Andrey Kravtsun added 4 commits April 7, 2017 01:49

Homework solved.

a676876

Unsuppress warnings on checked conversion (static ALWAYS_TRUE and ALW…

07081f4

…AYS_FALSE).

Removing test on private constructor for Collections.

1e86bda

Fixed warnings on previous commit's unused imports left.

a153e56

sproshev suggested changes Apr 10, 2017

View reviewed changes

Some fixes from pull request notes.

e29c717

sproshev approved these changes May 5, 2017

View reviewed changes

Final changes.

19e3471

		@@ -0,0 +1,42 @@
		package ru.spbau.mit;

		// extends Function1<T1, Function1<T2, S>>

		assertEquals(0, Collections.foldr(FAIL_FUNCTION, EMPTY_ARR, 0));

		}


		private static final Predicate<Object> NOT_NULL = IS_NULL.not();

		private static final List<Object> EMPTY_ARR = (List<Object>) emptyList();

Comments

Conversation

kravtsun commented Apr 6, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sproshev Apr 10, 2017 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

sproshev left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

kravtsun commented Apr 6, 2017 •

edited

Loading

sproshev Apr 10, 2017 •

edited

Loading